feat(PVO11Y-4761): Add Kanary signal exporter #574
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
from
d3779d6 to
6a403f0
Compare
|
Depends on #576 |
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
3 times, most recently
from
0afa481 to
27b8e81
Compare
|
/retest |
FaisalAl-Rayes
requested review from
raks-tt,
martysp21 and
pacho-rh
and removed request for
raks-tt,
martysp21 and
pacho-rh
martysp21
reviewed
May 27, 2025
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
2 times, most recently
from
e697cd6 to
23ba13a
Compare
FaisalAl-Rayes
requested review from
martysp21
and removed request for
kubasikus,
pacho-rh,
raks-tt and
martysp21
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
2 times, most recently
from
d9cfc2f to
857f5c0
Compare
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
from
857f5c0 to
042b9e2
Compare
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
2 times, most recently
from
5af0285 to
bd505e9
Compare
|
/retest |
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
3 times, most recently
from
4ba0f47 to
50efa28
Compare
|
@FaisalAl-Rayes I pushed a commit with various changes I made during review. I've set up a mtg tmrow for us to sync and discuss it :) |
kubasikus
reviewed
Jun 1, 2025
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
from
92f0ac2 to
252cfaf
Compare
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
2 times, most recently
from
bc02688 to
99a9c2e
Compare
kubasikus
reviewed
Jun 2, 2025
This commit introduces a new exporter to fetch Kanary signal values from a database and expose them as Prometheus metrics. Includes: - Core exporter logic for PostgreSQL - `kanary_up` metric with `instance` - `kanary_error` metric with `instance` and `reason` labels - Use a NewRegistry to control exposed Prometheus metrics. This ensures only explicitly registered application metrics are exported. - README.md Signed-off-by: Faisal Al-Rayes <[email protected]> # Conflicts: # go.mod
FaisalAl-Rayes
force-pushed
the
kanary-signal-exporter
branch
from
99a9c2e to
613d1ee
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit introduces a new exporter to fetch Kanary signal values from a database and expose them as Prometheus metrics.
Includes:
kanary_upmetric withinstancelabels.kanary_errormetric withinstanceandreasonlabels